Skip to content

chore: modernized demos with modern bundler setup and TypeScript#164

Merged
tomayac merged 3 commits intosqlite:mainfrom
jurerotar:chore/demo
Apr 16, 2026
Merged

chore: modernized demos with modern bundler setup and TypeScript#164
tomayac merged 3 commits intosqlite:mainfrom
jurerotar:chore/demo

Conversation

@jurerotar
Copy link
Copy Markdown
Contributor

I've modernized the demos a tiny bit, they now use Vite & TypeScript 😄

I've built this on top of the oxfmt PR, that's why its included in here as well 😄

Copy link
Copy Markdown
Collaborator

@tomayac tomayac left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@tomayac tomayac merged commit a81e4ef into sqlite:main Apr 16, 2026
8 checks passed
@solodeveloping
Copy link
Copy Markdown

It does look a lot more complete 👍

@solodeveloping
Copy link
Copy Markdown

I don't need it but, eventually, if someone else request it, they might be interested in more documentation about the exec method, which is rather complicated.

@sgbeal
Copy link
Copy Markdown
Collaborator

sgbeal commented Apr 21, 2026

Feel free make concrete suggestions for the exec docs. AFAIK every aspect of it is covered in the API docs. (Yes, it is too complicated - one of my definite design regrets in that API, but the alternative would have been to have different methods for different types of queries, which early development showed was even more confusing.)

@solodeveloping
Copy link
Copy Markdown

I would make a post somewhere if I had time, but it's time consuming and I have thousands of todos 😅

I figured out what I wanted, but:

  • resultRows: I was not clear on how to use it (maybe it's a ref that is filled and can be used to retrieve the results later? idk)
  • stmt: I haven't tried iit, so I don't know if it's difficult or not

I'd say some examples of the different options and interactions would help 🤷‍♂️ In the current state, it's one of these things where you have to try it for yourself, log the response and see what's going on

Personally, I don't need it, I'm done adapting to the new system (as far as I know)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants